Skip to content

chore: update spam list#2220

Merged
exploreriii merged 1 commit into
hiero-ledger:mainfrom
bhuvan-somisetty:fix/2217-update-spam-list
May 2, 2026
Merged

chore: update spam list#2220
exploreriii merged 1 commit into
hiero-ledger:mainfrom
bhuvan-somisetty:fix/2217-update-spam-list

Conversation

@bhuvan-somisetty

@bhuvan-somisetty bhuvan-somisetty commented May 1, 2026

Copy link
Copy Markdown
Contributor

Adds fauds-dev101 to the spam list as required in issue #2217.

Only .github/spam-list.txt was modified. No other changes included.

@bhuvan-somisetty
bhuvan-somisetty requested review from a team as code owners May 1, 2026 19:59
@codacy-production

codacy-production Bot commented May 1, 2026 •
edited
Loading

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@coderabbitai

coderabbitai Bot commented May 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Caution

Review failed

An error occurred during the review process. Please try again later.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 0282745e-32d2-4607-9804-ecf3aa941391

📥 Commits

Reviewing files that changed from the base of the PR and between 501def0 and a196465.

📒 Files selected for processing (1)
  • .github/spam-list.txt

Walkthrough

A single identifier, fauds-dev101, was appended to .github/spam-list.txt. No code, API, or public/exported declarations were modified.

Changes

Spam List Update

Layer / File(s) Summary
Data / Config
.github/spam-list.txt		 Appended new spam identifier fauds-dev101 as a new line (one-line addition).
Tests / Documentation
none		 No tests or documentation changed.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding an entry to the spam list file.
Description check ✅ Passed The description is clearly related to the changeset, explaining what was added and confirming only one file was modified.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

@MonaaEid MonaaEid linked an issue May 1, 2026 that may be closed by this pull request
Update spam list (1 additions, 0 removals) #2217
Closed
exploreriii
exploreriii previously approved these changes May 1, 2026
View reviewed changes
@exploreriii exploreriii added the step: merge ready PR is ready and waiting to merge label May 1, 2026
@exploreriii

exploreriii commented May 1, 2026

Copy link
Copy Markdown
Contributor

Hi please update branch

@aceppaluni

aceppaluni commented May 1, 2026

Copy link
Copy Markdown
Contributor

@bhuvan-somisetty

This is looking great. Looks like the pre-commit is not passing. You can run this locally to see issues before pushing. If you need any assistance with this please let us know :)

@bhuvan-somisetty

bhuvan-somisetty commented May 1, 2026

Copy link
Copy Markdown
Contributor Author

Hi, I have updated the branch as requested and synced with latest main. Please review and approve workflows when possible. Thanks!

MonaaEid
MonaaEid requested changes May 1, 2026
View reviewed changes
Comment thread .github/spam-list.txt Outdated
exploreriii
exploreriii requested changes May 2, 2026
View reviewed changes

@exploreriii exploreriii left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @bhuvan-somisetty you lost your sign status

@bhuvan-somisetty
chore: update spam list
a196465 
Signed-off-by: Bhuvan Somisetty <somisettybhuvan5@gmail.com>
@bhuvan-somisetty
bhuvan-somisetty force-pushed the fix/2217-update-spam-list branch from da1d976 to a196465 Compare May 2, 2026 14:21
@bhuvan-somisetty

bhuvan-somisetty commented May 2, 2026

Copy link
Copy Markdown
Contributor Author

Hi, I have updated the PR as per the requested changes.
The update is limited to .github/spam-list.txt for issue #2217, with correct formatting and a signed commit.
It seems the workflow failure is due to a YAML syntax issue in request-triage-review.yml and not related to this PR.
Kindly re-review when possible. Thank you!

@MonaaEid
MonaaEid requested a review from exploreriii May 2, 2026 19:41
@bhuvan-somisetty

bhuvan-somisetty commented May 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

@exploreriii
Please could you review this.
And lemme know if any changes required

@exploreriii
exploreriii merged commit a534436 into hiero-ledger:main May 2, 2026
13 checks passed
@manishdait manishdait added this to the v0.2.6 milestone May 4, 2026
NssGourav pushed a commit to NssGourav/hiero-sdk-python that referenced this pull request May 14, 2026
@bhuvan-somisetty @NssGourav
chore: update spam list (hiero-ledger#2220)
1c8cd24 
Signed-off-by: Bhuvan Somisetty <somisettybhuvan5@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MonaaEid MonaaEid MonaaEid approved these changes
@manishdait manishdait manishdait approved these changes
@exploreriii exploreriii exploreriii approved these changes
@nadineloepfe nadineloepfe Awaiting requested review from nadineloepfe nadineloepfe is a code owner automatically assigned from hiero-ledger/hiero-sdk-python-maintainers
@rbarker-dev rbarker-dev Awaiting requested review from rbarker-dev rbarker-dev is a code owner automatically assigned from hiero-ledger/github-maintainers

Assignees

No one assigned

Labels

step: merge ready PR is ready and waiting to merge

Projects

None yet

Milestone

v0.2.6

Development

Successfully merging this pull request may close these issues.

Update spam list (1 additions, 0 removals)

5 participants

@bhuvan-somisetty @exploreriii @aceppaluni @MonaaEid @manishdait